@Yoni-Starkware reviewed 5 of 6 files at r1, all commit messages.
Reviewable status: 5 of 6 files reviewed, 1 unresolved discussion

crates/apollo_starknet_os_program/src/cairo/starkware/starknet/core/os/block_context.cairo line 89 at r1 (raw file):

            sequencer_address=0,
        ),
        starknet_os_config=starknet_os_config,

Keep it without the * in the block context struct, to make the access to chain_id and fee_token_address cheaper

Suggestion:

starknet_os_config=[starknet_os_config],

@noaov1 reviewed 5 of 6 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @meship-starkware)

crates/apollo_starknet_os_program/src/cairo/starkware/starknet/core/os/block_context.cairo line 60 at r2 (raw file):

    execute_deprecated_syscalls_ptr: felt*,
    compiled_class_facts_bundle: CompiledClassFactsBundle*,
    public_keys_hash: felt,

Is this needed?

Code quote:

    public_keys_hash: felt,

@Yoni-Starkware reviewed 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @meship-starkware)

Reviewable status: 3 of 6 files reviewed, 2 unresolved discussions (waiting on @noaov1 and @Yoni-Starkware)

crates/apollo_starknet_os_program/src/cairo/starkware/starknet/core/os/block_context.cairo line 89 at r1 (raw file):

Done.

crates/apollo_starknet_os_program/src/cairo/starkware/starknet/core/os/block_context.cairo line 60 at r2 (raw file):

Done.

@noaov1 reviewed 3 of 3 files at r3, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Yoni-Starkware)

@noaov1 reviewed 2 of 2 files at r5, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Yoni-Starkware)

@Yoni-Starkware reviewed 2 of 3 files at r3, 1 of 1 files at r4, 2 of 2 files at r5, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @meship-starkware)